Skip to content

Added ability to change relative paths according to path to root .less file. (#119)#291

Open
Nodge wants to merge 19 commits intoleafo:masterfrom
Nodge:master
Open

Added ability to change relative paths according to path to root .less file. (#119)#291
Nodge wants to merge 19 commits intoleafo:masterfrom
Nodge:master

Conversation

@Nodge
Copy link
Copy Markdown

@Nodge Nodge commented Aug 17, 2012

Support an extra parameter to change relative paths in url() statement. Enabled by default.

@leafo
Copy link
Copy Markdown
Owner

leafo commented Aug 17, 2012

Why re-parse the url out of what's already been parsed and compiled?

@travisbot

This comment has been minimized.

@andrew13
Copy link
Copy Markdown

I like this as well. Would be nice to have it in the package rather than manually editing the file.

@neilime
Copy link
Copy Markdown

neilime commented Nov 19, 2012

Does this support of relative links will be integrated one day ?

@stof
Copy link
Copy Markdown
Contributor

stof commented Jan 3, 2013

note that less.js 1.3.2 implemented a similar feature

neilime and others added 4 commits January 24, 2013 18:41
: allows url with params (file.css?param=sample)
Improving the ability to change relative paths according to path to root
…oot:

Allow to manually set base url path (useful for generating css on the fly)
@neilime
Copy link
Copy Markdown

neilime commented Feb 22, 2013

This pull request should be merged, ability to change relative paths according to root path, is very usefull to manage & parse assets efficiently

Comment thread lessc.inc.php Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By default, it should not rewrite to be consistent with the behavior of latest less.js

behavior of latest less.js
Add setter to define allowUrlRewrite
Set allowUrlRewrite default to false to be consistent with the behavior of latest less.js
@neilime
Copy link
Copy Markdown

neilime commented Feb 25, 2013

Url rewriting option is false by default now, (as less.js), so this pull request don't change the default behavior.

@neilime
Copy link
Copy Markdown

neilime commented May 7, 2013

Is there any chance that this PR will be accepted, or should we found an another solution (if exists) to manage relative path ?

Repository owner deleted a comment from travisbot Oct 15, 2018
Repository owner deleted a comment from travisbot Oct 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants